Skip to content

Conversation

stamhankar999
Copy link
Contributor

  • Update docs.yaml to exclude an appveyor url from external link checking. The link is good, but documentor erroneously thinks the link is bad.

* Update docs.yaml to exclude an appveyor url from external link checking. The link is good, but documentor erroneously thinks the link is bad.
@mikefero mikefero changed the title * Document how to run unit and integration tests Document how to run unit and integration tests Dec 16, 2016
Copy link
Contributor

@mikefero mikefero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, but have a couple comments/thoughts for discussion.

# Testing
Before proceeding ensure the tests were built using the [build procedures].

Both unit and integration tests use [Boost.Test](http://www.boost.org/doc/libs/1_62_0/libs/test/doc/html/index.html)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 on the description; however should we also indicate how to launch them in Windows?

# Run a particular suite
test/unit_tests/cassandra_unit_tests -t blacklist_lb
```

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • There is also a configuration file (that is being deprecated) in favor of command line driven options, we might also want to include information about the config.txt file until it is fully removed and that it should be located in the working directory (integration tests only as the unit tests do not require any additional configuration elements to run). I don't think we should document the defaults as we update them during release to use the latest Cassandra version at the time.
  • CCM information is also missing as a dependency (this is required for the integration tests to run properly)

Copy link
Contributor

@mikefero mikefero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@stamhankar999 stamhankar999 merged commit 34a95f6 into master Jan 24, 2017
@stamhankar999 stamhankar999 deleted the test_doc branch January 24, 2017 17:11
mpenick pushed a commit that referenced this pull request Dec 10, 2019
CPP-833 - Port named parameters tests to Google test framework
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants